sizegroup: Use for loops

For loops to loop over lists look nicer and actually do the right thing
with "break" and "continue" statements. So they are vastly preferred to
while loops.

sizegroups: Restructure code

This way, we do the checks at the start of the effected function, not
before calling it.

sizegroup: Don't cache the sizes anymore

This simplifies code and because sizes are cached by the widgets
themselves, it's not a large performance problem (unless people use huge
amounts of widgets in a single size group, but who does that?

Merge branch 'bgo687196-filesystemmodel-crash'

bgo#687196 - Fix model corruption during file removal

The main problem is that we were emitting the row-deleted signal for the model in the middle
of the process that actually deletes the row from the model (remove the row from the array,
update the model->file_lookup hash table, etc.).  In the model's caller, one of the row-deleted
callbacks was requesting an iter, which caused the model to revalidate itself - but it did
this while it was in an inconsistent state.  This led to an assertion failure later when the
model resorted itself.

The fix in remove_file() is like this:

* The filteredness/visibility of the deleted node is not updated.  The
  node will simply be gone; we don't need to update those values at
  all.

* We invalidate just the node that is being deleted.

* The model->file_lookup hash table is not completely nuked; instead,
  we carefully adjust its indices.

* The row-deleted signal is only emitted at the very end, when
  deletion is complete and the model is consistent.

Many thanks to William Hua for doing the detective work on this bug!

Signed-off-by: Federico Mena Quintero <federico@gnome.org>

Make freeze_updates() and thaw_updates() static functions

They were in the semi-public API of GtkFileSystemModel, but never actually used outside of it.

Signed-off-by: Federico Mena Quintero <federico@gnome.org>

Comments on how the filtering and sorting processes work

Signed-off-by: Federico Mena Quintero <federico@gnome.org>

sizegroup: Always at least use widget's size

When widgets were hidden, they were otherwise assigned a 0 size.

settings: Reset all styles when the enable-animations settings changes

https://bugzilla.gnome.org/show_bug.cgi?id=686021

Raleigh: Fix spinners with disabled animations

When animations are disabled, active and inactive spinners should look
different.

https://bugzilla.gnome.org/show_bug.cgi?id=686021

cssimage: Implement some equal functions

Remove argument to _gtk_file_system_model_update_file() that should not be part of the internal API

Signed-off-by: Federico Mena Quintero <federico@gnome.org>

Rename gtk_tree_path_new_from_node() to tree_path_new_from_node()

This is a function internal to the file system model; let's not pollute the gtk_tree_path namespace.

Also, make the 'i' variable into 'r' as it refers to a row index, not a file-array index (for
consistency with the docs and the rest of the code).

Signed-off-by: Federico Mena Quintero <federico@gnome.org>

icon-theme: support loading symbolic GFileIcons from generic URIs

Right now we support loading and recoloring symbolic GFileIcons, but
only if the underlying GFile has a local path. This breaks when the
GFileIcon is loaded from a GResource, which is a reasonable option for an
application that wants to ship a custom symbolic icon.

This patch changes GtkIconInfo to store a GFile together with the file
path, and changes the symbolic icon lookup code to use the GFile URI,
which transparently makes the code work also for GResources.

https://bugzilla.gnome.org/show_bug.cgi?id=687059

themingbackground: Remove struct members

... and put them in the only function they are used in.

reftests: Add reftest for fractional border sizes

themingengine: Draw fradctional border sizes

reftests: Fix linear-gradient reftest

... and add it to the Makefile

themingbackground: Remove GtkThemingBackgroundLayer

The struct was just the index. So just pass the index around instead of
a full struct.

themingbackground: Use get_box() for background image size

themingbackground: Introduce gtk_theming_background_get_box()

to query the different clip boxes used by the background drawing code.
Use this function to query these boxes when clipping.

themingbackground: Add content_box variable

... to go with border_box and padding_box.

themingbackground: Get rid of flags variable

themingbackground: Move image variable

... from the Layer struct onto the stack of the only function using it.

themingbackground: Restructure code some more

Move variable initialization outside the first code with side effects.
This allows adding some more early returns, including one for code that
used to trigger g_return_if_fail() in certain corner cases.

themingbackground: Restructure code

Make if statements encompassing the whole function into early returns.
The rest of the diff is reindenting.

cssimage: Add a warning for drawing empty images

width and height of an image must be > 0 for the image to get drawn.
It's up to the code further up to ensure that this is not happening.

reftests: Add a reftest for recent commit

Check that a computed background-size of 0 is treated as 0.

cssvalue: Compute "background-size: 0 0" properly

Previously a computed value of 0 was treated as "auto", which is wrong.

stylecontext: Don't use bg image in gtk_style_context_set_background()

Old code tried to use the "background-image" proeprty for setting the
default image background. While this used to work in the early days of
GTK3, today it is grossly misleading as the backgronud image may be
resized, repositioned and semi-translucent which causes very weird
artifacts when rendering.

So we use the background-color only instead.

cssimage: Add an equal vfunc

No implementations for it exist yet.

menuitem: Draw background unconditionally

settings: Use _gtk_css_provider_load_named()

This way we create one provider per settings object instead of stuffing
it into a global unchanging never-deleting hash table.
Also, we now reload the theme when instructed instead of keeping the old
loaded (and possibly stale) data forever.

https://bugzilla.gnome.org/show_bug.cgi?id=683896

cssprovider: Move fallback code into _gtk_css_provider_load_named()

This makes sure the full theme loading logic resides in one function and
isn't scattered around.

As a side-effect, the hash table kept by gtk_css_provider_get_named()
will now be populated with fallback themes. This will not be a problem
after the next commit though.

cssprovider: Export gtk_css_provider_load_named) function

... and document it.
For now, the function is only exported internally.

cssprovider: Split out theme loading function

Split maintaining the global themes hash table and the theme loading
code into two functions.
This also fixes leaking the provider when loading a theme from a builtin
resource.

Updated Galician translations

Updated Marathi Translations

Updated German translation

[l10n] Updated Estonian translation

Updated Slovenian translation

Updated Lithuanian translation

wayland: update to work with stable libxkbcommon

libxkbcommon has had some changes to its API. However, it now has a
stable release (0.2.0), so this makes the necessary changes, and
replaces all uses of the deprecated API.

Signed-off-by: Ran Benita <ran234@gmail.com>

Updated Spanish translation

Updated Indonesian translation

Updated gujarati file

Updated Serbian translation

Fix compiling for win32

https://bugzilla.gnome.org/show_bug.cgi?id=687066

texthandles: Add an extra style class to the cursor-mode handle

Themes may want to render handles differently depending on whether
the widget is in selection mode (2 handles enclosing a selection) or
cursor mode (one handle pointing out the insertion cursor).

texthandles: set input shape on handles' window

This improves both interaction and theming, as it allows
arbitrary handle shapes while just being draggable from
the visible areas.

This way themes can set up handles with the hotspot visually
displaced from the horizontal center, as long as the hotspot
lies centered in the image/svg asset.

texthandle: Fix shape setup on non-composited environments

The check on the handle to be drawn on the mask was based on the yet to
be set priv->windows pointers, pass explicitly the handle position to
have the shape correctly initialized on non-composited environments

Assamese translation updatedas.po

Fix typo from f2ab3af

Updated Latvian translation

configure.ac: Look for wayland-client 1.0.0

Add GDK_VERSION_3_8

This macro is needed for the new AVAILABLE_IN and DEPRECATED_IN
macros.

Updated Greek translation

Bump version

3.7.0

Be robust against unrealized windows in GtkWindowAccessible

Based on a patch by Albert Astals Cid,
https://bugzilla.gnome.org/show_bug.cgi?id=686152

Forgotten fixup

This was meant to be included in the previous commit :-(

treeview: Add support for styling the dragged header

https://bugzilla.gnome.org/show_bug.cgi?id=684980

treeview: Move the dragged header in the headers window

https://bugzilla.gnome.org/show_bug.cgi?id=684980

Use named union for _GtkSymbolicColor in gtk/gtksymboliccolor.c

Patch by Richard Lloyd,
https://bugzilla.gnome.org/show_bug.cgi?id=686366

scrollbar: remove unused variable

My bad.

notebook: return TRUE for drag-motion event when over tabs

The GtkNotebook drag-motion event handler may install a timeout when
hovering over a tab, in order to switch to it.
On the other hand it's desirable for applications to use the empty tab
area as a drop target, so the drag-motion handler returns FALSE
(also in case it installs the switch tab timeout), as explained in
https://bugzilla.gnome.org/show_bug.cgi?id=350665.

Unfortunately, applications can use the tab label widget (or a child
of it) as a different drop target area, and install their own
drag-motion handler there.
In this scenario, the timeout will still be installed by GtkNotebook's
handler, but since it returns FALSE, it will never get the matching
drag-leave event, causing it to trigger also when the mouse pointer
moved elsewhere before it expired.

Fix this by returning TRUE from drag-motion when the event is over a
tab. Note that this makes automatic tab switching not work anymore when
drag and drop is handled in the tab label widget; applications are
expected to also handle tab switching if desired in such a case.

https://bugzilla.gnome.org/show_bug.cgi?id=684415

notebook: consolidate code to remove the switch tab timer

https://bugzilla.gnome.org/show_bug.cgi?id=684415

[l10n] Updated Estonian translation

scale: update style properties on GtkRange at init

Same fix as in ef027c93d49e2e31ccb8c181d0096c705c141906, but for
the GtkScale subclass of GtkRange.

https://bugzilla.gnome.org/show_bug.cgi?id=686280

scrollbar: update style properties on GtkRange at init

GtkScrollbar used to rely on style-updated being emitted every time
after the widget was created in order to set the right values from its
style properties on GtkRange.
Nowadays we try to be smarter and avoid emitting style-updated at
creation time, so we need to manually initialize the GtkRange values.

This fixes a regression from 35e36b9fe506143f722a00f26a86143f9c95b740.

https://bugzilla.gnome.org/show_bug.cgi?id=686280

Fix out of source tree building for gdk/broadway

This patch fix compilation when building out of source tree (gdk/broadway dir).

Signed-off-by: Colin Walters <walters@verbum.org>

Updated Slovak translation

Updated Galician translations

docs: fix typo on 'Getting Started with Gtk+' page

Updated Lithuanian translation

Update Czech translation

wayland: Update to reflect protocol changes

Update Galician translations

Updated Slovenian translation

Fix duplicate columns in filechooser entry completion

This partically reverts commit
331bba1ad6c5d9535fc5a827b91019f5824e25a7, which broke documented
behaviour.

menubutton: don't refer to the non-existant menu property

7c6454246e7fa181d9e16f3520bc6d3765168f40 removed the property, but
forgot to change the name in a g_object_notify().

Updated Serbian translation

Updated Indonesian translation

Updated Spanish translation

level-bar: Fix typo

scrolledwindow: set GDK_EXPOSURE_MASK on the overshoot window

Currently we use gtk_style_context_set_background() when the state flags
change in order to propagate the background color to the overshoot
window, but this is actually only needed because the window doesn't get
expose events, since we always draw a full background in draw().
This also fixes some problems when the GdkWindow of the scrolled
window's child is composited, as seen in oxygen-gtk3.

https://bugzilla.gnome.org/show_bug.cgi?id=686265

level-bar: add support for RTL locales

https://bugzilla.gnome.org/show_bug.cgi?id=684288

level-bar: add an "inverted" property like GtkProgressBar

https://bugzilla.gnome.org/show_bug.cgi?id=684288

Add gdk version macros for 3.8

https://bugzilla.gnome.org/show_bug.cgi?id=684288

GtkMenuButton: remove menu property as it is replaced by popup.

See that it was already announced to be removed before 3.6.0
but we forgot.

cssshadowsvalue: handle gtk_css_value_transition returning NULL

The implementation of transition for GtkCssShadowValue can return NULL
at least when the two values have a different inset; all other parts of
the GTK/CSS machinery (e.g. GtkCssArrayValue) handle this by returning
NULL too. Instead, GtkCssShadowsValue was returning an invalid value,
where "len" was set, but some values in the array were NULL, which would
lead to a segfault when this value is later evaluated by the compute
function.

Fix this by making GtkCssShadowsValue return NULL if a shadow transition
fails, like GtkCssArrayValue does.

https://bugzilla.gnome.org/show_bug.cgi?id=686013

reftests: fix the linear-gradient reftest

Don't use a repeating linear gradient, since it can't be easily
reftested against a non-repeating one for the reasons described in the
test header.
Instead, add a separate test for repeating gradients (against another
repeating gradient).

This makes the test pass, so it can be added to the Makefile now.

cssshadow: plug a cairo_surface_t leak

We were never destroying the cairo surface we use for blurring, which
would lead to a huge leak.

https://bugzilla.gnome.org/show_bug.cgi?id=686209

csskeyframes: unref GtkCssValues when free-ing

We assume a reference to all the GtkCssValues we store, so we need to
release it when free-ing.

gradient: plug a GtkCssValue refleak

Since _gtk_symbolic_color_resolve_full() returns a reference to a
GtkCssValue.

csscomputedvalues: plug a refleak

_gtk_css_keyframes_compute() returns a reference to a GtkCssKeyframes,
and _gtk_css_animation_new() takes another reference.

bump version

3.6.1

Fix css parser tests

Parsing a shorthand background property was running into unexpected
errors when trying position values where there were none. To fix this,
introduce a try_parse variant of the position parse function that
silently returns NULL.

Fix 'make check'